Automate zppy tests #520
Automate zppy tests #520
Conversation
tests/scripts/test_dev.bash
Outdated
| exit 1 | ||
| fi | ||
|
|
||
| exit 7 |
There was a problem hiding this comment.
Exiting here for now, since the subsequent steps are difficult to automate.
|
@chengzhuzhang This is an initial attempt at automating the steps from https://e3sm-project.github.io/zppy/_build/html/main/dev_guide/release_testing.html -- specifically just the There's a number of issues in fully automating this workflow. See the #TODO in-line comments. |
tests/integration/utils.py
Outdated
| if __name__ == "__main__": | ||
| generate_cfgs(unified_testing=False) | ||
| generate_cfgs( | ||
| unified_testing=(sys.argv[1] == "True"), diags_environment_commands=sys.argv[2] |
There was a problem hiding this comment.
Should be args 2 and 3
|
Made some adjustments post-zppy/zppy-interfaces split. Can't run these changes yet because need to get around 1) issues with conda activation in a script and 2) automating the waiting for multiple jobs to finish. |
| fi | ||
| git reset --hard upstream/main | ||
| conda clean --all --y | ||
| # TODO: previous iterations of this script had issues with activating conda environments |
There was a problem hiding this comment.
Probably need the conda sourcing command too: e.g., source /gpfs/fs1/home/ac.forsyth2/miniforge3/etc/profile.d/conda.sh
|
|
||
| # Set up zppy-interfaces env | ||
| cd ${ZI_DIR} | ||
| git fetch upstream main |
There was a problem hiding this comment.
Even simple lines like this are going to require user input on systems that require GitHub passphrases every time fetch is run.
| ZPPY_DIR=/home/ac.forsyth2/zppy/ | ||
| DIAGS_DIR=/home/ac.forsyth2/e3sm_diags/ | ||
| ZI_DIR=/home/ac.forsyth2/zppy-interfaces/ |
There was a problem hiding this comment.
Or we need to have the script do git clone fresh...
|
#709 replaces the work in this PR. |
Automate zppy tests. Resolves #516.